Skip to content

Conversation

@yaauie
Copy link

@yaauie yaauie commented Sep 3, 2025

supercedes #2

high-level:

  • Every ${THING}Bridge is a top-level interface extending StableBridgeAPI<>, accessed exclusively through one of:
    • static factory methods (e.g., ${THING}Bridge.create(...)) (no constructors).
    • public AbstractExternal${BRIDGE} implements ${BRIDGE} for things that have expected external implementations.
  • The ProxyInternal classes are all effectively private implementation details. Simple implementations are still nested in the interface that they implement, but more complex ones are their own top-level classes.

Copy link
Owner

@mashhurs mashhurs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@mashhurs mashhurs merged commit abe7548 into mashhurs:logstash-bridge-geoip-interfaces Sep 3, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants